Skip to content

Ard terravision - #596

Open
Pratichhya wants to merge 8 commits into
mainfrom
ard_terravision
Open

Ard terravision#596
Pratichhya wants to merge 8 commits into
mainfrom
ard_terravision

Conversation

@Pratichhya

@Pratichhya Pratichhya commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

First version of including two UDPs implemented as a part of the Terravision project: https://terravision-project.eu/
These include simple indices calculations that are used within the project, namely:

  • BSI
  • AMWI

related PR: ITA-TECNOLOGIA/Ard-Terravision#2

@Pratichhya
Pratichhya requested a review from JanssenBrm July 13, 2026 16:38
@Pratichhya Pratichhya linked an issue Jul 13, 2026 that may be closed by this pull request
@Pratichhya

Copy link
Copy Markdown
Contributor Author

I think now it is failing mostly due to a timeout. @JanssenBrm in the meantime, please let me know if you have any suggestion or feedback

@algorithm-services-catalogue

Copy link
Copy Markdown

🔍 Catalogue's Preview Site Deployed

Your changes have been deployed to the preview site:

🔗 Preview URL: https://esa-apex.github.io/apex-algorithms-catalogue-web/pr-preview/pr-596/

This preview will be updated automatically when you push new changes to your PR.

@JanssenBrm

Copy link
Copy Markdown
Contributor

Hi @Pratichhya,

This looks very nice, I just did a small fix to the formatting of the amwi record as it was causing issues. Added a test to prevent this in the future.

My only comment on is that the UDP description is quite "basic". It is not really explaining what the index is about and how to interpret the values in the result. It would be great if these could be added as this will improve the service record quite a bit.

@JanssenBrm JanssenBrm left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See previous comment

@Pratichhya

Copy link
Copy Markdown
Contributor Author

@JanssenBrm thank you for the check. I updated the description in benchmark json files to reflect in the main page

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Integration of TerraVision Spectral Indices

2 participants